Skip to content

Task done - #144

Open
bagochus wants to merge 1 commit into
katrendev:masterfrom
bagochus:master
Open

Task done#144
bagochus wants to merge 1 commit into
katrendev:masterfrom
bagochus:master

Conversation

@bagochus

Copy link
Copy Markdown

Добрый день, отправляю решение тестового.
-Для гарантированного закрытия файла реализовал интерфейс IDisposable в ReadOnlyStream, однако ответственность за правильный вызов Dispose или использование через using остается на вызывающей стороне
-LetterStats я бы реализовал через класс, а не структуру, это бы позволило использовать более компактные методы
-В описании к функции Main ничего не сказано об удалении гласных/согласных из статистики, однако в коде оно имеется. Решил что код главнее, поэтому реализовал функцию и подправил описание.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant